Postfix calculator by Palezehvat · Pull Request #3 · Palezehvat/HomeworksCSharp

Palezehvat · 2023-03-08T18:07:21Z

Постфиксный калькулятор

…eworksCSharp into PostfixCalculator

yurii-litvinov · 2023-03-09T16:14:25Z

BWT/BWT/Program.cs

@@ -1,208 +0,0 @@
-namespace Sort;


Этот коммит удалит домашку с BWT. Если это и был план, то это всё равно незачем делать в этом пуллреквесте, удалите отдельно.

Напоминаю. Может, Вам за неё стыдно, но не всё так плохо ведь :)

yurii-litvinov · 2023-03-09T16:15:23Z

stackCalculator/stackCalculator/Program.cs

@@ -0,0 +1,259 @@
+namespace Sort;


Корневое пространство имён проекта по традиции называется так же, как сам проект, а раз так, то и сам проект (и решение) называется в PascalCase.

yurii-litvinov · 2023-03-09T16:16:06Z

stackCalculator/stackCalculator/Program.cs

+
+using System;
+
+interface OperationsWithElementsStruct


В C# есть правило "одна сущность — один файл", так что этот интерфейс с очень странным называнием надо вынести в отдельный файл, переименовать и написать к нему комментарий.

yurii-litvinov · 2023-03-09T16:16:28Z

stackCalculator/stackCalculator/Program.cs

+
+interface OperationsWithElementsStruct
+{
+    // Add element to struct


struct — это другое, Вы не понимаете :) В условии речь шла про стек

yurii-litvinov · 2023-03-09T16:17:10Z

stackCalculator/stackCalculator/Program.cs

+    bool IsEmpty();
+}
+
+public class StackWithArray : OperationsWithElementsStruct


И к самому классу нужен комментарий

yurii-litvinov · 2023-03-09T16:27:24Z

stackCalculator/stackCalculator/Program.cs

+                int multiplier = 10;
+                int number = 0;
+                while (i < stringWithExpression.Length && stringWithExpression[i] >= '0' && stringWithExpression[i] <= '9')
+                {
+                    number += stringWithExpression[i] - '0';
+                    number *= multiplier;
+                    ++i;
+                }
+                number /= 10;
+                --i;
+                if (isNeedMinus)
+                {
+                    number *= -1;
+                }


Погуглите string.Split() и int.TryParse :)

yurii-litvinov · 2023-03-09T16:28:07Z

stackCalculator/stackCalculator/Program.cs

+            {
+                double numberAfter = 0;
+                (var isCorrect, var firstNumber) = stackExpression.RemoveElement();
+                (isCorrect, var secondNumber) = stackExpression.RemoveElement();


Так первый isCorrect перетрётся

yurii-litvinov · 2023-03-09T16:28:27Z

stackCalculator/stackCalculator/Program.cs

+        {
+            return (false, 0);
+        }
+        return true == stackExpression.IsEmpty() ? (true, result) : (false, 0);


true == , кажется, бессмысленно

yurii-litvinov · 2023-03-09T16:28:47Z

stackCalculator/stackCalculator/Program.cs

+
+class Program
+{
+    public static void Main(string[] args)


Используйте top-level statements

Не поправлено

yurii-litvinov · 2023-03-09T16:30:01Z

stackCalculator/stackCalculator/Program.cs

+    // Receives the input string in which the expression is written in postfix form, finds the result
+    public (bool, double) ConvertToAResponse(string stringWithExpression)
+    {
+        StackList stackExpression = new StackList();


Стековый калькулятор должен знать только про интерфейс стека (то есть вообще в коде класса «Стековый калькулятор» не должно быть ни одного упоминания конкретных реализаций стека, даже если очень хочется).

Это невыполнение требований условия.

…ь они идут отдельно с маленькой, при этом в каталоге отображаются с большой, а при добавлении с большой git status их не выделяет, поэтому этот комит их отдельное добавление

Другая ветка

yurii-litvinov · 2023-03-13T11:53:08Z

BWT/BWT/Program.cs

@@ -1,208 +0,0 @@
-namespace Sort;


Напоминаю. Может, Вам за неё стыдно, но не всё так плохо ведь :)

yurii-litvinov · 2023-03-13T11:53:43Z

stackCalculator/stackCalculator/InterfaceForStack.cs

@@ -0,0 +1,16 @@
+namespace StackCalculator;
+
+ interface IOperationsWithStack


Лишний пробел перед interface и фигурной скобкой ниже

И нужен комментарий к самому интерфейсу

yurii-litvinov · 2023-03-13T11:54:22Z

stackCalculator/stackCalculator/Program.cs

+
+class Program
+{
+    public static void Main(string[] args)


Не поправлено

yurii-litvinov · 2023-03-13T11:54:34Z

stackCalculator/stackCalculator/Program.cs

+using System;
+class Program


Надо пустую строку

yurii-litvinov · 2023-03-13T11:54:42Z

stackCalculator/stackCalculator/Program.cs

+{
+    public static void Main(string[] args)
+    {
+        Test test = new Test();


Не пишите тип дважды

yurii-litvinov · 2023-03-13T11:57:09Z

stackCalculator/stackCalculator/StackCalculator.cs

+
+public class StackCalculator
+{
+    private double delta = 0.0000000000001;


Suggested change

private double delta = 0.0000000000001;

private const double delta = 0.0000000000001;

yurii-litvinov · 2023-03-13T11:58:10Z

stackCalculator/stackCalculator/StackCalculator.cs

+    // Receives the input string in which the expression is written in postfix form, finds the result
+    public (bool, double) ConvertToAResponse(string stringWithExpression)
+    {
+        Stack stackExpression = new();


Stack — это конкретный класс, так что требования условия всё ещё не выполняются. Хорошая попытка, но должно быть наоборот, ArrayStack наследует от интерфейса Stack

yurii-litvinov · 2023-03-13T11:58:26Z

stackCalculator/stackCalculator/StackCalculator.cs

+        string[] expressionArray = stringWithExpression.Split(' ');
+        while (i < expressionArray.Length)
+        {
+            var isCorrectNumber =  Int32.TryParse(expressionArray[i], out var number);


Suggested change

var isCorrectNumber = Int32.TryParse(expressionArray[i], out var number);

var isCorrectNumber = Int32.TryParse(expressionArray[i], out var number);

yurii-litvinov · 2023-03-13T11:59:56Z

stackCalculator/stackCalculator/StackWithList.cs

+        public double valueStack { get; set; }
+        public StackElement next { get; set; }


Всё, что public, в C# пишется с заглавной

yurii-litvinov · 2023-03-13T12:00:04Z

stackCalculator/stackCalculator/Tests.cs

+    // Tests the program
+    public bool TestForProgram()
+    {
+        StackCalculator calculator = new StackCalculator();


Suggested change

StackCalculator calculator = new StackCalculator();

var calculator = new StackCalculator();

yurii-litvinov

Не-а, это всё равно удалит BWT, попробуйте ещё раз

palorel and others added 4 commits March 4, 2023 16:59

+Stack

0628989

Delete BWT directory

52b35a6

+Тесты +дельта +2 реализация стека

077d251

Merge branch 'PostfixCalculator' of https://github.com/Palezehvat/Hom…

a035abf

…eworksCSharp into PostfixCalculator

yurii-litvinov suggested changes Mar 9, 2023

View reviewed changes

palorel and others added 4 commits March 10, 2023 22:20

Разбиение на файлы

7c32bd2

Реализация бора

b539f54

Delete Bor directory

a14870b

Другая ветка

yurii-litvinov suggested changes Mar 13, 2023

View reviewed changes

palorel added 2 commits March 15, 2023 15:33

Ревью2

28f0687

Для BWT создал новую ветку и перенёс его туда, чтобы откаты не делать

fb95603

yurii-litvinov suggested changes Mar 16, 2023

View reviewed changes

Нашёл ошибку пока писал модульные тесты, исправил

07208f9

		@@ -0,0 +1,16 @@
		namespace StackCalculator;

		interface IOperationsWithStack

	private double delta = 0.0000000000001;
	private const double delta = 0.0000000000001;

	var isCorrectNumber = Int32.TryParse(expressionArray[i], out var number);
	var isCorrectNumber = Int32.TryParse(expressionArray[i], out var number);

		public double valueStack { get; set; }
		public StackElement next { get; set; }

	StackCalculator calculator = new StackCalculator();
	var calculator = new StackCalculator();

Conversation

Palezehvat commented Mar 8, 2023

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yurii-litvinov left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Comments